Skip to content

Get CI running again, and install locked dependencies - #42

Merged
livid merged 2 commits into
mainfrom
fix-ci
Nov 11, 2026
Merged

livid merged 2 commits into
mainfrom
fix-ci

Conversation

@livid

@livid livid commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Two commits on top of main:

  • Install locked, hashed dependencies in the image. The image installs requirements.lock with --require-hashes. make lock refreshes it.
  • Get CI running again. The workflow pinned 2021 action and hook versions that no longer work. Actions move to their current major versions, the hooks run through plain pre-commit, and isort, black and bandit move to the oldest release that still installs. The test job moves to Python 3.14, the version the image uses.

This pull request exists to run the workflow before it lands on main, where a green run also publishes the image.

🤖 Generated with Claude Code

livid and others added 2 commits November 10, 2026 19:28
The image now installs requirements.lock with --require-hashes instead of whatever is newest on the day of the build. `make lock` re-resolves requirements.txt inside the image's own Python and rewrites the lock.

The lock holds the 68 versions the production container already runs, so this changes no behaviour.

Verified: an image built from the lock has the same pip freeze as the running container and passes the 59 tests; a lock with a wrong hash fails the build.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
The workflow pinned 2021 versions that no longer work: pre-commit/action used a cache API GitHub has retired, and the isort, black and bandit hooks no longer install into a fresh environment. Actions move to their current major versions, the hooks run through plain pre-commit, and the three hooks move to the oldest release that still installs (no file is reformatted).

The test job moves to Python 3.14, the version the image uses. The bandit assert skip now matches the path form current bandit reports.

Verified locally: all hooks pass from an empty pre-commit cache. Not verified locally: the workflow itself.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@codecov-commenter

codecov-commenter commented Nov 11, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.39%. Comparing base (5cdaff1) to head (f0fb101).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   91.18%   91.39%   +0.21%     
==========================================
  Files          19       19              
  Lines        1373     1395      +22     
  Branches      161       81      -80     
==========================================
+ Hits         1252     1275      +23     
- Misses         91       92       +1     
+ Partials       30       28       -2     
Flag Coverage Δ
unittests 91.39% <ø> (+0.21%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@livid
livid merged commit f0fb101 into main Nov 11, 2026
3 checks passed
@livid
livid deleted the fix-ci branch November 11, 2026 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants